Conversation
📝 WalkthroughWalkthroughThe PR bumps the package to 4.5.1 and centralizes the Changes
Sequence Diagram(s)(Skipped — changes are URL/path updates and do not introduce new multi-component control flow.) Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Fix all issues with AI agents
In `@src/tools/client.ts`:
- Around line 8-19: The BASE_URL currently uses process.env.OCTOMIND_API_URL
directly which can already include a trailing "/api", causing double "/api" when
createClient is called; update the BASE_URL computation (the constant named
BASE_URL used when creating client via createClient<paths>) to detect and remove
a trailing "/api" (or trailing slash + "api") from process.env.OCTOMIND_API_URL
if present, falling back to "https://app.octomind.dev" when unset, so the final
baseUrl passed to createClient is always single "/api" appended.
In `@src/tools/test-cases.ts`:
- Around line 130-132: The URL construction in src/tools/test-cases.ts currently
concatenates BASE_URL with a path string which can produce duplicate "/api"
segments when BASE_URL already contains "/api"; update the code that builds the
URL (the const url = new URL(...) usage) to call new URL with the path as the
first argument and BASE_URL as the second (e.g., new
URL("/api/bearer/v1/test-targets/${options.testTargetId}/test-cases", BASE_URL))
so the URL constructor normalizes the path regardless of whether BASE_URL
already includes "/api". Ensure you reference BASE_URL and options.testTargetId
in the replacement to preserve behavior.
Summary by CodeRabbit
Chores
Documentation
Tests
✏️ Tip: You can customize this high-level summary in your review settings.